[18.0] [MIG] delivery_postlogistics: Migration to 18.0 #974
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Superseeds #940 |
This was referenced Feb 10, 2025
StephaneMangin
force-pushed
the
18.0-mig-delivery_postlogistics
branch
5 times, most recently
from
98eb1c9 to
4d44284
Compare
|
hello @StephaneMangin |
|
/ocabot migration delivery_postlogistics |
3 tasks
jbaudoux
reviewed
Feb 14, 2025
delivery_postlogistics/tests/fixtures/cassettes/test_token_error.yaml
Outdated
Show resolved
Hide resolved
delivery_postlogistics/tests/fixtures/cassettes/test_store_label.yaml
Outdated
Show resolved
Hide resolved
delivery_postlogistics/tests/fixtures/cassettes/test_store_label.yaml
Outdated
Show resolved
Hide resolved
delivery_postlogistics/tests/fixtures/cassettes/test_missing_language.yaml
Outdated
Show resolved
Hide resolved
delivery_postlogistics/tests/fixtures/cassettes/test_missing_language.yaml
Outdated
Show resolved
Hide resolved
jbaudoux
reviewed
Feb 17, 2025
bizzappdev
reviewed
Feb 17, 2025
Yes it is transitional, because these values are added in a dependency. Updating... |
StephaneMangin
force-pushed
the
18.0-mig-delivery_postlogistics
branch
4 times, most recently
from
3fe8f87 to
78c780f
Compare
yankinmax
reviewed
Feb 20, 2025
StephaneMangin
force-pushed
the
18.0-mig-delivery_postlogistics
branch
2 times, most recently
from
0f171db to
894894d
Compare
jbaudoux
reviewed
Feb 25, 2025
jbaudoux
reviewed
Feb 25, 2025
StephaneMangin
force-pushed
the
18.0-mig-delivery_postlogistics
branch
3 times, most recently
from
e9dc13c to
0b24d58
Compare
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: delivery-carrier-16.0/delivery-carrier-16.0-delivery_postlogistics Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_postlogistics/
Postlogistics limit for fields such as name, street and city, is 35 chars for the recipient fields but is limited to 25 characters.
Sometimes due to API error it may happen we don't receive anything back when requesting token. In this case we want to show meaningful message.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: delivery-carrier-16.0/delivery-carrier-16.0-delivery_postlogistics Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_postlogistics/
If a stock.picking is created and in the same transaction we generate a label, when the webservice returns an error, the module rollbacks the transaction, so 'self' is empty and will fail with: Record does not exist or has been deleted. (Record: stock.picking(8211067,), User: 6) When trying to write the tracking number, and will therefore not raise the details of the error.
The tests from d5c3f73 failed due to partner city being required
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: delivery-carrier-16.0/delivery-carrier-16.0-delivery_postlogistics Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_postlogistics/
StephaneMangin
force-pushed
the
18.0-mig-delivery_postlogistics
branch
7 times, most recently
from
22b6efe to
cc1636f
Compare
StephaneMangin
force-pushed
the
18.0-mig-delivery_postlogistics
branch
from
cc1636f to
683d7fa
Compare
leemannd
reviewed
Mar 18, 2025
| res["postlogistics_default_package_type_id"] = self.env.ref( | ||
| "delivery_postlogistics.postlogistics_default_package_type", | ||
| raise_if_not_found=False, | ||
| ).id |
There was a problem hiding this comment.
The raise_if_not_found will raise a Nonetype if the xmlid is not found
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Includes:
postlogistics_send_shippingrequirements (Using ECO default shipping pricing)Depends on:
To avoid Odoo external URL calls restriction, decorate your test class with
@common.tagged("-standard")(removing the standard tag which is by default). This way, Odoo doesn't patch therequest.Session.sendmethod to throw an exception in an external URL call case.